Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
Contributor
WalkthroughThis PR adds a new blog announcement for the Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 1 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 3
🧹 Nitpick comments (2)
apps/blog/content/blog/try-prisma-announcment-Kv6bwRcdjd/index.mdx (2)
25-25: Consider hyphenating "up to date" as "up-to-date".When used as a compound adjective before a noun, "up-to-date" should be hyphenated for clarity.
✍️ Proposed style fix
-For a while now, we at Prisma have maintained our [prisma-examples](https://github.com/prisma/prisma-examples) repository with up to date, ready-to-go projects using Prisma. The intent for these projects is to allow users to experience what Prisma would be like in their new stack while evaluating technologies. +For a while now, we at Prisma have maintained our [prisma-examples](https://github.com/prisma/prisma-examples) repository with up-to-date, ready-to-go projects using Prisma. The intent for these projects is to allow users to experience what Prisma would be like in their new stack while evaluating technologies.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@apps/blog/content/blog/try-prisma-announcment-Kv6bwRcdjd/index.mdx` at line 25, Replace the phrase "up to date" with the hyphenated compound adjective "up-to-date" in the sentence referencing the prisma-examples repository (the text containing "prisma-examples" and "ready-to-go projects") so it reads "up-to-date, ready-to-go projects" for correct compound-adjective usage.
96-96: Consider more concise phrasing: "all the" instead of "all of the".For brevity, "all the options" is more concise than "all of the options."
✍️ Proposed style fix
-If you are curious about all of the options available via the CLI, head over to the [`try-prisma` repository](https://github.com/prisma/try-prisma) and check out the README. +If you are curious about all the options available via the CLI, head over to the [`try-prisma` repository](https://github.com/prisma/try-prisma) and check out the README.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@apps/blog/content/blog/try-prisma-announcment-Kv6bwRcdjd/index.mdx` at line 96, Replace the phrase "all of the options available via the CLI" with the more concise "all the options available via the CLI" in the sentence that links to the try-prisma repository (the line containing "If you are curious about all of the options available via the CLI, head over to the [`try-prisma` repository"](https://github.com/prisma/try-prisma) and check out the README."); update that sentence text accordingly to use "all the options" for brevity.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@apps/blog/content/blog/try-prisma-announcment-Kv6bwRcdjd/index.mdx`:
- Line 3: Update the slug value "try-prisma-announcment-Kv6bwRcdjd" to
"try-prisma-announcement-Kv6bwRcdjd" in the MDX frontmatter and rename the
containing folder from try-prisma-announcment-Kv6bwRcdjd to
try-prisma-announcement-Kv6bwRcdjd; also search for and update any references to
the old slug/folder (routing, links, sitemap, imports) so the generated URL
becomes /try-prisma-announcement-... consistently.
- Line 18: The line beginning with an unmatched opening double-quote before the
sentence about the CLI utility (the text containing "try-prisma" and `npx
try-prisma`) is malformed; fix it by removing the stray leading double-quote or
adding a matching closing double-quote so the sentence is properly
punctuated—update the content in the MDX entry (the paragraph mentioning
try-prisma) to have balanced quotation marks or no extraneous quote.
In `@apps/blog/src/app/layout.tsx`:
- Line 39: The body padding removed in layout.tsx breaks the blog TOC's sticky
positioning—either restore the 6rem padding by adding back pt-24 on the <body>
className in layout.tsx, or update the TOC styles that currently use "sticky
top-24" and "max-h-[calc(100vh-6rem)]" to compute offsets from the real header
height (e.g. replace top-24 and calc(100vh-6rem) with a CSS variable or concrete
header-height value like top-[var(--header-height)] and max-h-[calc(100vh -
var(--header-height))] in the TOC component) so the sticky TOC no longer depends
on body padding.
---
Nitpick comments:
In `@apps/blog/content/blog/try-prisma-announcment-Kv6bwRcdjd/index.mdx`:
- Line 25: Replace the phrase "up to date" with the hyphenated compound
adjective "up-to-date" in the sentence referencing the prisma-examples
repository (the text containing "prisma-examples" and "ready-to-go projects") so
it reads "up-to-date, ready-to-go projects" for correct compound-adjective
usage.
- Line 96: Replace the phrase "all of the options available via the CLI" with
the more concise "all the options available via the CLI" in the sentence that
links to the try-prisma repository (the line containing "If you are curious
about all of the options available via the CLI, head over to the [`try-prisma`
repository"](https://github.com/prisma/try-prisma) and check out the README.");
update that sentence text accordingly to use "all the options" for brevity.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 3b0ff4d5-9687-49d2-bc3d-d78cd14e0f49
⛔ Files ignored due to path filters (5)
apps/blog/public/authors/sabin-adams.pngis excluded by!**/*.pngapps/blog/public/try-prisma-announcment-Kv6bwRcdjd/imgs/hero.svgis excluded by!**/*.svgapps/blog/public/try-prisma-announcment-Kv6bwRcdjd/imgs/meta.pngis excluded by!**/*.pngapps/blog/public/try-prisma-announcment-Kv6bwRcdjd/imgs/process.gifis excluded by!**/*.gifapps/blog/public/try-prisma-announcment-Kv6bwRcdjd/imgs/suggestions.pngis excluded by!**/*.png
📒 Files selected for processing (3)
apps/blog/content/blog/rethinking-database-migrations/index.mdxapps/blog/content/blog/try-prisma-announcment-Kv6bwRcdjd/index.mdxapps/blog/src/app/layout.tsx
apps/blog/content/blog/try-prisma-announcment-Kv6bwRcdjd/index.mdx
Outdated
Show resolved
Hide resolved
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Actionable comments posted: 2
♻️ Duplicate comments (1)
apps/blog/content/blog/try-prisma-announcment-Kv6bwRcdjd/index.mdx (1)
3-3:⚠️ Potential issue | 🟠 MajorFix the slug typo to avoid SEO and asset-path inconsistency.
announcmentis misspelled in the slug and repeated in image paths. This creates a non-ideal canonical URL and bakes the typo into content asset links. Please rename toannouncementconsistently (including the folder name and any references).Proposed fix (frontmatter + in-file paths)
-slug: "try-prisma-announcment-Kv6bwRcdjd" +slug: "try-prisma-announcement-Kv6bwRcdjd" ... -metaImagePath: "/try-prisma-announcment-Kv6bwRcdjd/imgs/meta.png" -heroImagePath: "/try-prisma-announcment-Kv6bwRcdjd/imgs/hero.svg" +metaImagePath: "/try-prisma-announcement-Kv6bwRcdjd/imgs/meta.png" +heroImagePath: "/try-prisma-announcement-Kv6bwRcdjd/imgs/hero.svg" ... - + ... - +Also applies to: 9-10, 67-67, 92-92
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@apps/blog/content/blog/try-prisma-announcment-Kv6bwRcdjd/index.mdx` at line 3, The slug value "try-prisma-announcment-Kv6bwRcdjd" contains a typo; rename the slug to "try-prisma-announcement-Kv6bwRcdjd" and update every in-file reference of "announcment" to "announcement" (this includes the frontmatter slug field and all image/asset paths where the misspelling appears). Also rename the containing folder to match the corrected slug and verify links within the document (search for the token "announcment" and fix occurrences noted around lines 9-10, 67, and 92). Ensure consistency so canonical URL and asset paths use "announcement".
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@apps/blog/content/blog/try-prisma-announcment-Kv6bwRcdjd/index.mdx`:
- Line 25: Replace the non-hyphenated compound adjective "up to date" with the
hyphenated form "up-to-date" in the sentence that currently reads "with up to
date, ready-to-go projects..." inside the index.mdx content so it becomes "with
up-to-date, ready-to-go projects..."; update that exact phrase in the file to
apply proper hyphenation.
- Line 8: Update the frontmatter metaDescription value to correct the grammar by
replacing "an example projects" with either "an example project" (singular) or
"example projects" (plural); locate the metaDescription entry in the frontmatter
of the file and edit the string to read, for example, "Announcing: Try Prisma. A
CLI utility that allows you to easily set up and try out an example project
using Prisma." referencing the metaDescription key in the document.
---
Duplicate comments:
In `@apps/blog/content/blog/try-prisma-announcment-Kv6bwRcdjd/index.mdx`:
- Line 3: The slug value "try-prisma-announcment-Kv6bwRcdjd" contains a typo;
rename the slug to "try-prisma-announcement-Kv6bwRcdjd" and update every in-file
reference of "announcment" to "announcement" (this includes the frontmatter slug
field and all image/asset paths where the misspelling appears). Also rename the
containing folder to match the corrected slug and verify links within the
document (search for the token "announcment" and fix occurrences noted around
lines 9-10, 67, and 92). Ensure consistency so canonical URL and asset paths use
"announcement".
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: b8b3b3d2-769e-4629-b46b-7c166ece060d
📒 Files selected for processing (1)
apps/blog/content/blog/try-prisma-announcment-Kv6bwRcdjd/index.mdx
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary by CodeRabbit
New Features
Documentation
Style